-
-
Notifications
You must be signed in to change notification settings - Fork 529
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
i3808 missing Fortran compiler config for macOS M1 runner #3811
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3811 +/- ##
========================================
Coverage 99.59% 99.59%
========================================
Files 257 257
Lines 20806 20806
========================================
Hits 20722 20722
Misses 84 84 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see any issues with this if everything is working. Have both updated workflows been tested on your fork already?
If so I will switch this to an approval
I did not create this PR from a branch through my fork. The PR tests are working here since the erroneous cached wheel is removed prior during the |
From the workflow run, I forgot that |
Ok if I forget to come back and check if it worked, just ping me |
Triggered another run for the scheduled tests here: https://github.com/pybamm-team/PyBaMM/actions/runs/7835843395 |
…m#3811) * pybamm-team#3808 install missing macOS deps for M1 runner * pybamm-team#3808 fix 360-minute timeouts for `pybamm-requires` compilation * pybamm-team#3808 ensure `pip` never uses `scikits.odes` wheels * style: pre-commit fixes * Don't run `pybamm_install_odes` on Python 3.12 macOS M1 --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Description
Closes #3808
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.
Key checklist:
$ pre-commit run
(or$ nox -s pre-commit
) (see CONTRIBUTING.md for how to set this up to run automatically when committing locally, in just two lines of code)$ python run-tests.py --all
(or$ nox -s tests
)$ python run-tests.py --doctest
(or$ nox -s doctests
)